Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix missing quote and syntax highlight in README #938

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Legend-Master
Copy link
Contributor

What does this PR do?

Fix some missing quotes and syntax highlighting in README

Before

image

After

image

@coveralls
Copy link

coveralls commented Nov 26, 2023

Coverage Status

coverage: 84.254%. remained the same
when pulling 18456cd on Legend-Master:readme-code-block-fix
into 1e3d5ce on redhat-developer:main.

Copy link
Collaborator

@gorkem gorkem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@pierreprinetti
Copy link
Contributor

Hey @Legend-Master
I am seeing this after posting #945.

Your solution is changing the examples to make them JSON, while I believe that they were written to be YAML. Can you please have a look at #945 and see if it makes sense?

@Legend-Master
Copy link
Contributor Author

I think they're json files used by code editors, not yaml files (but since yaml contains/allows all json things, syntax highlighting should still works)

image

@pierreprinetti
Copy link
Contributor

I think they're json files used by code editors, not yaml files

Oh! Interesting.

@pierreprinetti
Copy link
Contributor

@gorkem PTAL

@msivasubramaniaan
Copy link
Contributor

@Legend-Master Please resolve the merge conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants